-
Notifications
You must be signed in to change notification settings - Fork 43
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding instascale e2e test #271
Adding instascale e2e test #271
Conversation
63764d6
to
c12863a
Compare
e53cae5
to
bed345d
Compare
242cf34
to
c8a97a2
Compare
58dc44b
to
544a0b5
Compare
503cfcd
to
88d1ceb
Compare
027e028
to
63aa183
Compare
63aa183
to
c133dba
Compare
497a5cc
to
6a65959
Compare
6a65959
to
14c5e48
Compare
14c5e48
to
a34ea19
Compare
@astefanutti @dimakis Can you please take a look? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very well done! Just left of couple of comments, otherwise looks very good!
test/support/codeflare.go
Outdated
OsdClusterID = "CLUSTERID" | ||
// Determine if test is being run on an OSD cluster, used for testing InstaScale. | ||
IsOSD = "IS_OSD" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could it be assumed that it's an OSD cluster if an OSD cluster ID has been provided?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes I think so, I have updated to reflect this.
Thanks for the review @astefanutti ! I have pushed some changes if you could take another look when you have time please. |
0cf71d1
to
115ecd6
Compare
115ecd6
to
ff040e3
Compare
/lgtm Great work! |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: astefanutti The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Issue link
Closes project-codeflare/instascale#66
What changes have been made
An e2e test has been added to test the entire instascale flow on an OSD Cluster.
Verification steps
make all-in-one
these should already be present. NFD and ClusterPolicy.Other Considerations
Checks